-
Notifications
You must be signed in to change notification settings - Fork 7.8k
drivers: misc: Add Low-voltage detection (LVD) driver support for RX130 #93694
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
drivers: misc: Add Low-voltage detection (LVD) driver support for RX130 #93694
Conversation
The following west manifest projects have changed revision in this Pull Request:
✅ All manifest checks OK Note: This message is automatically posted and updated by the Manifest GitHub Action. |
6018f8c
to
c11e903
Compare
drivers/misc/renesas_rx_lvd/Kconfig
Outdated
config RENESAS_RX_LVD_INIT_PRIORITY | ||
int "RX LVD initialization priority" | ||
default KERNEL_INIT_PRIORITY_DEVICE | ||
help | ||
System initialization priority for RX LVD driver. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be wrapped by RENESAS_RX_LVD
config RENESAS_RX_LVD_INIT_PRIORITY | |
int "RX LVD initialization priority" | |
default KERNEL_INIT_PRIORITY_DEVICE | |
help | |
System initialization priority for RX LVD driver. | |
if RENESAS_RX_LVD | |
config RENESAS_RX_LVD_INIT_PRIORITY | |
int "RX LVD initialization priority" | |
default KERNEL_INIT_PRIORITY_DEVICE | |
help | |
System initialization priority for RX LVD driver. | |
endif # RENESAS_RX_LVD |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's done
* SPDX-License-Identifier: Apache-2.0 | ||
*/ | ||
|
||
/* Local Includes */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please drop this
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I removed this
Update commit ID for hal_renesas for adding LVD support Signed-off-by: Quy Tran <quy.tran.pz@renesas.com>
Add header file to support Renesas LVD APIs on Zephyr Signed-off-by: Quy Tran <quy.tran.pz@renesas.com>
Introduce Low/Programmable voltage detection driver for Renesas RX which using r_lvd_rx from RDP. The target is to monitor the voltage level input to Vcc/CMPA2 pin using a program Signed-off-by: Quy Tran <quy.tran.pz@renesas.com>
Add LVD node support on RX130 Signed-off-by: Quy Tran <quy.tran.pz@renesas.com>
Add a simple sample for LVD driver on RSK-RX130-512kb Signed-off-by: Quy Tran <quy.tran.pz@renesas.com>
c11e903
to
d012894
Compare
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Introduce low-voltage detection driver support on RX130
The voltage detection circuit (LVD) monitors the voltage level input to the VCC or external pin using a program.
Upon detecting a voltage level transition (either upward or downward) past the configured threshold, the system can generate a reset or an interrupt, enabling appropriate protective or recovery actions.